Skip to content

Add support for managing tags#14

Open
gavinsanders wants to merge 1 commit into
redhataccess:masterfrom
gavinsanders:insights_tags
Open

Add support for managing tags#14
gavinsanders wants to merge 1 commit into
redhataccess:masterfrom
gavinsanders:insights_tags

Conversation

@gavinsanders
Copy link
Copy Markdown

The access-insights client supports tags configured via a tags.yaml file. This creates the tags.yaml file.
Addditionally stdlib is added as a requirement to use the to_yaml function.

Comment thread manifests/current.pp
}
}

file { "/etc/${package_name}/tags.yaml":
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll admit I don't know too much about the insights client (my knowledge is mainly on the Puppet side). Is this something that should be there before it's registered?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From what I've heard it will be picked up by the agent, but not directly. If you want it directly you have to unregister/register the client again. That is for the file-redaction.yaml file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants